Skip to content
This repository has been archived by the owner on May 5, 2023. It is now read-only.

Add ServiceRuntime sample #152

Merged
merged 4 commits into from Apr 17, 2012
Merged

Add ServiceRuntime sample #152

merged 4 commits into from Apr 17, 2012

Conversation

markcowl
Copy link
Member

Sample shows:

(1) Retrieving local config
(2) Getting endpoint details for the local instance
(3) Writing to the DiagnosticStore LocalResource

var RoleEnvironment = azure.RoleEnvironment;

var LogFileKey = "DiagnosticLog";
var DefaultLogFileName = "DiagnisticLog.txt";

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Diagnostic ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@andrerod
Copy link

Added a few comments mark. Sorry i missed this PR the other day. Could you go through the feedback ? Thank you.

@markcowl
Copy link
Member Author

Thanks for the feedback! Working on deployment this evening, will go through your comments Monday

Sent from my Windows Phone


From: André Rodrigues
Sent: 4/14/2012 11:29 AM
To: Mark Cowlishaw
Subject: Re: [azure-sdk-for-node] Add ServiceRuntime sample (#152)

Added a few comments mark. Sorry i missed this PR the other day. Could you go through the feedback ? Thank you.


Reply to this email directly or view it on GitHub:
#152 (comment)

@markcowl
Copy link
Member Author

Addressed review comments, please comment or merge.

@andrerod
Copy link

Looks good.

andrerod pushed a commit that referenced this pull request Apr 17, 2012
Add ServiceRuntime sample
@andrerod andrerod merged commit 53994cc into Azure:dev Apr 17, 2012
andrerod pushed a commit that referenced this pull request Jun 7, 2012
andrerod pushed a commit that referenced this pull request Jun 7, 2012
#152: Fixing a bunch of JSON outputs.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants